Conversation
Gives ability to add a wrapper around the sidebar and the active step. Useful for styling.
|
This looks great! Looking forward to see this feature in the Repo :) |
AndersGerner
left a comment
AndersGerner
left a comment
There was a problem hiding this comment.
Just to give my thumbs up - I think it looks great :)
There are things that needs to be updated like existBeforeEnter on AnimatePresence, but that is out of scope for this PR. I would like to see this soon in the next version :)
|
Thanks @AndersGerner for the thumbs up. I haven't dug into the codebase long enough to know about the existBeforeEnter and AnimatePresence yet, but sounds like something I can do. But until @devrnt approves the PR, I'm just patching the package in my project so I can use the steps. Also, when I was using it in my current project, I ended up not using the I considered removing the sidebarAndStepWrapper from this PR, but decided against it as others might find it useful. |
2 participants
Added sidebar and sidebarAndStepWrapper prop.
This will enable the consumer to implement a sidebar based off the steps rather than hard coding the steps in a separate sidebar component.